Skip to content
This repository was archived by the owner on Dec 17, 2025. It is now read-only.

Remove KotlinPoet from Project#946

Closed
Lysander wants to merge 18 commits intomasterfrom
chausknecht/remove-kotlinpoet
Closed

Remove KotlinPoet from Project#946
Lysander wants to merge 18 commits intomasterfrom
chausknecht/remove-kotlinpoet

Conversation

@Lysander
Copy link
Collaborator

@Lysander Lysander commented Oct 13, 2025

This PR removes the dependencies to kotlinpoet from the project.

The LensesAnnotationProcessor internally now works without and relies on simple StringBuilder-functionality. The readability is sometimes worse, but also better in some places, as it resembles more the final created code structure.

Nevertheless, this is a huge step for being able to upgrade to new Kotlin versions in much quicker time!

@fixes #936

@Lysander Lysander added this to the 1.0-RC21 milestone Oct 13, 2025
@Lysander Lysander requested a review from haukesomm October 13, 2025 13:42
@Lysander Lysander self-assigned this Oct 13, 2025
@Lysander
Copy link
Collaborator Author

Type aliases are not working anymore.

We need to add some tests and repair the naming resolution for those!

@Lysander Lysander marked this pull request as draft October 14, 2025 13:11
@Lysander
Copy link
Collaborator Author

After further investigations, it seems, that the version of kotlinpoet does NOT affect our upgrade schedule!

So this is probably the false approach and should be closed, if #947 will work!

@Lysander
Copy link
Collaborator Author

Because of #947 we do not need this and it can be closed.

@Lysander Lysander closed this Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Try to remove kotlinpoet from the dependencies

1 participant